Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release changes #178

Merged
merged 108 commits into from
Jul 17, 2023
Merged

Release changes #178

merged 108 commits into from
Jul 17, 2023

Conversation

anayeaye
Copy link
Collaborator

@anayeaye anayeaye commented May 17, 2023

Warning
When deploying these changes to operations stacks that assume a data access role, make sure the new environmental configuration variables VEDA_RASTER_AWS_REQUEST_PAYER and VEDA_RASTER_EXPORT_ASSUME_ROLE_CREDS_AS_ENVS are updated in the secrets manager.

Changes

Dashboard-facing changes

  • Titiler and titiler-pgstac upgrades
    • Custom mosaic tiler factory no longer needed (ships with upgrade)
    • Standardized band names are now always b1, b2, etc. Previous versions of titiler used numeric band names for some COGs. This is visible in the /cog/statistics and cog/info results
  • Fix to /cog/* endpoints' json handling of no data values
  • Option to deploy tiler with requester-pays configuration

Backend updates

  • Temporary fix inserts assumed role credentials in raster-api environment Tiling/AWS Session problems in veda-dev #192, Export AWS Access Credentials If Needed #194
  • Python upgrade to 3.9
  • RDS public accessibility now configurable
  • Optional RDS proxy- Local and test database loading script (supports both automated tests and local runs)
  • Bulk database modification sql execution tools in support scripts capture methods used for the correction of float values in stac projection extension
  • Experimental /collection-id-search endpoint searches for all items meeting filter criteria and returns unique ids (may be updated/replaced by stac api aggregation extension). Long term support is not committed for this endpoint.

moradology and others added 30 commits January 4, 2023 14:00
Update deploying_to_existing_environments.md
…da environment to agree to pay all S3 transfer costs
@anayeaye anayeaye marked this pull request as ready for review June 13, 2023 18:02
@anayeaye anayeaye changed the title [WIP] Release changes Release changes Jun 13, 2023
@anayeaye anayeaye requested review from slesaad, ividito and smohiudd June 20, 2023 16:38
@@ -1,11 +1,11 @@
FROM lambci/lambda:build-python3.8
FROM public.ecr.aws/sam/build-python3.9:latest
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
FROM public.ecr.aws/sam/build-python3.9:latest
FROM --platform=linux/amd64 public.ecr.aws/sam/build-python3.9:latest

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i've had issues with deploying from the new mac, this solves it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might as well do it for all the Dockerfiles

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wait, not just mac, but also from github actions 🤔

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even with export DOCKER_DEFAULT_PLATFORM=linux/amd64? This has been deploying from github actions as is. I'm reading up, though, to see if we should take out a branch from develop to make the change.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't github actions running on ubuntu so shouldn't need --platform=linux/amd64?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thought so too, maybe i am misremembering. but it's always a good idea to add it whereever.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with that. I've run into so many problems with this when running locally on my mac.

router=APIRouter(route_class=LoggerRouteHandler),
)
app.include_router(stac.router, tags=["Items"], prefix="/stac")

cog = TilerFactory(
router_prefix="/cog",
optional_headers=optional_headers,
gdal_config=settings.get_gdal_config(),
environment_dependency=settings.get_gdal_config,
Copy link
Contributor

@smohiudd smohiudd Jul 4, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So environment_dependency is a callable and not a dict anymore in titiler. That was one of the breaking changes when gdal_config was removed:

https://github.com/developmentseed/titiler/blob/047453fced0cf9d7b7e43c08a4ba4e4a569d9e04/src/titiler/core/titiler/core/factory.py#L157.

Here are the change notes:

https://github.com/developmentseed/titiler/blob/main/CHANGES.md#070-2022-06-08

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants